Skip to content

Conversation

@phoevos
Copy link
Member

@phoevos phoevos commented Dec 2, 2025

Add the following model version tags when logging a model to MLflow:

  • model_uri: The URI of the model artifact
  • model_type: The type of the model (e.g. 'medcat_snomed')
  • validation_status: The validation status of the model (e.g. 'pending')

Add the following model version tags when logging a model to MLflow:
* model_uri: The URI of the model artifact
* model_type: The type of the model (e.g. 'medcat_snomed')
* validation_status: The validation status of the model (e.g. 'pending')

Signed-off-by: Phoevos Kalemkeris <phoevos.kalemkeris@ucl.ac.uk>
@phoevos phoevos added the enhancement New feature or request label Dec 2, 2025
Signed-off-by: Phoevos Kalemkeris <phoevos.kalemkeris@ucl.ac.uk>
Copy link
Member

@baixiac baixiac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Just make sure that versions[0] will always return the most recently saved model version.

(versions[0] had been used prior to this PR and should have been tested)

model_name: str,
model_manager: ModelManager,
validation_status: str = "pending",
model_type: Optional[str] = None,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All CMS models have the ModelType, hence there's no need to make the argument optional and mlflow.set_tag() will not set a None value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants